Skip to content

Conversation

@ki-met-hoon
Copy link
Collaborator

Pull request

Related issue

Resolve #118
Resolve #130

Motivation and context

  • 채팅 메시지 카프카 발행 로직
  • 카프카로부터 메시지 수신 로직

Solution

  • 채팅 메시지 전달 시 채팅방 ID를 키 값으로 전달해 메시지 순서 보장
    • 실시간 성이 떨어짐
    • 더 나은 방법이 없는지 고민해보겠습니다.
  • 각 채팅 서버 별로 컨슈머 그룹 설정
  • 파티션 수에 맞게 멀티 컨슈머 스레드 설정
  • 보람님 추가 기능 구현
    • 로컬 메모리와 유저 ID를 비교하는 로직, 있으면 웹소켓을 통한 데이터 전달 없으면 일단 버림
    • 웹소켓 전달하는 로직 or 전달하는 함수
    • ChatMessageToKafka DTO 수정

How has this been tested

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the docs/CONTRIBUTING.md document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@ki-met-hoon ki-met-hoon added ✨ Feature 기능 추가 📦 Environment 개발 환경 세팅 🎮 BE 백엔드 ♻️ Need To Refactor 추후 리팩토링이 필요한 이슈 및 논의 무조건 스프린트내에 해야하는 것들 labels Feb 4, 2025
@ki-met-hoon ki-met-hoon added this to the 주톡피아 마일스톤2 milestone Feb 4, 2025
@ki-met-hoon ki-met-hoon self-assigned this Feb 4, 2025
@bo-ram-bo-ram
Copy link
Member

LGTM

@ki-met-hoon ki-met-hoon merged commit 37ecb3a into dev Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🎮 BE 백엔드 📦 Environment 개발 환경 세팅 ✨ Feature 기능 추가 ♻️ Need To Refactor 추후 리팩토링이 필요한 이슈 및 논의 무조건 스프린트내에 해야하는 것들

Projects

None yet

Development

Successfully merging this pull request may close these issues.

채팅 메시지 카프카 전달 및 수신 기능 구현 채팅 서버 주식 당일일분봉 데이터 Consumer 구현

3 participants